Skip to content

fix: make sql query PostgreSQL compatible#8163

Merged
klesh merged 1 commit intoapache:mainfrom
Tonkonozhenko:patch-1
Nov 6, 2024
Merged

fix: make sql query PostgreSQL compatible#8163
klesh merged 1 commit intoapache:mainfrom
Tonkonozhenko:patch-1

Conversation

@Tonkonozhenko
Copy link
Copy Markdown
Contributor

@Tonkonozhenko Tonkonozhenko commented Oct 29, 2024

Summary

Make SQL queries work in PostgreSQL

@dosubot dosubot Bot added size:S This PR changes 10-29 lines, ignoring generated files. component/plugins This issue or PR relates to plugins pr-type/bug-fix This PR fixes a bug labels Oct 29, 2024
@Tonkonozhenko Tonkonozhenko changed the title Fix sql string escaping fix: make sql query PostgreSQL compatible Oct 29, 2024
Copy link
Copy Markdown
Contributor

@klesh klesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
Would you like to create another PR to the branch release-v1.0 so it can be released sooner?

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Oct 30, 2024
@Tonkonozhenko
Copy link
Copy Markdown
Contributor Author

@klesh done: #8167

@klesh klesh merged commit 3a7acd6 into apache:main Nov 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component/plugins This issue or PR relates to plugins lgtm This PR has been approved by a maintainer pr-type/bug-fix This PR fixes a bug size:S This PR changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants